From e248a8c4cae6a533b43ffcd8130277c192ea91d2 Mon Sep 17 00:00:00 2001 From: Eli Zaretskii Date: Sat, 21 Mar 2009 11:29:28 +0000 Subject: [PATCH] inc/sys/stat.h (struct stat): Change the types of st_uid and st_gid to unsigned. inc/pwd.h (struct passwd): Change the types of pw_uid and pw_gid to unsigned. (getpwuid): Argument is now unsigned. uid_t is now unsigned. ntlib.c (setuid): Argument is now unsigned. (getuid): Return value is now unsigned. (getpwuid): Argument is now unsigned. (fchown): UID and GID arguments are now unsigned. ntlib.h (fchown): UID and GID arguments are now unsigned. (getuid): Return value is now unsigned. (setuid): Argument is now unsigned. (getpwuid): Remove prototype (it's declared in nt/inc/pwd.h). w32.c (getpwuid): Change argument type to unsigned. (struct w32_id): Change type of `rid' member to unsigned. (w32_cached_id, w32_add_to_cache, get_name_and_id): Change type of argument ID to unsigned. All callers changed. (getuid, geteuid, getgid, getegid): Change return type to unsigned. --- lib-src/ntlib.c | 8 ++++---- lib-src/ntlib.h | 7 +++---- nt/inc/pwd.h | 20 ++++++++++---------- nt/inc/sys/stat.h | 4 ++-- src/w32.c | 22 +++++++++++----------- 5 files changed, 30 insertions(+), 31 deletions(-) diff --git a/lib-src/ntlib.c b/lib-src/ntlib.c index 4ca7ae40aa1..bce4ce10f31 100644 --- a/lib-src/ntlib.c +++ b/lib-src/ntlib.c @@ -119,20 +119,20 @@ cuserid (char * s) return name; } -int +unsigned getuid () { return 0; } int -setuid (int uid) +setuid (unsigned uid) { return 0; } struct passwd * -getpwuid (int uid) +getpwuid (unsigned uid) { return NULL; } @@ -185,7 +185,7 @@ getpass (const char * prompt) } int -fchown (int fd, int uid, int gid) +fchown (int fd, unsigned uid, unsigned gid) { return 0; } diff --git a/lib-src/ntlib.h b/lib-src/ntlib.h index 4fe492241dd..7f9313c9ec4 100644 --- a/lib-src/ntlib.h +++ b/lib-src/ntlib.h @@ -35,11 +35,10 @@ char *getwd (char *dir); int getppid(void); char * getlogin (); char * cuserid (char * s); -int getuid (); -int setuid (int uid); -struct passwd * getpwuid (int uid); +unsigned getuid (); +int setuid (unsigned uid); char * getpass (const char * prompt); -int fchown (int fd, int uid, int gid); +int fchown (int fd, unsigned uid, unsigned gid); #ifndef BSTRING #define bzero(b, l) memset(b, 0, l) diff --git a/nt/inc/pwd.h b/nt/inc/pwd.h index 31c0df7ec30..d8a4ffc2e20 100644 --- a/nt/inc/pwd.h +++ b/nt/inc/pwd.h @@ -5,21 +5,21 @@ */ struct passwd { - char *pw_name; - char *pw_passwd; - int pw_uid; - int pw_gid; - int pw_quota; - char *pw_gecos; - char *pw_dir; - char *pw_shell; + char *pw_name; + char *pw_passwd; + unsigned pw_uid; /* Vista's TrustedInstaller has a very large RID */ + unsigned pw_gid; + int pw_quota; + char *pw_gecos; + char *pw_dir; + char *pw_shell; }; -typedef int uid_t; +typedef unsigned uid_t; typedef uid_t gid_t; struct passwd * getpwnam (char *); -struct passwd * getpwuid (int); +struct passwd * getpwuid (unsigned); #endif /* _PWD_H_ */ diff --git a/nt/inc/sys/stat.h b/nt/inc/sys/stat.h index a7c4c02ee06..4d93430803c 100644 --- a/nt/inc/sys/stat.h +++ b/nt/inc/sys/stat.h @@ -61,8 +61,8 @@ struct stat { dev_t st_dev; unsigned short st_mode; short st_nlink; - int st_uid; - int st_gid; + unsigned st_uid; /* Vista's TrustedInstaller has a very large RID */ + unsigned st_gid; unsigned __int64 st_size; dev_t st_rdev; time_t st_atime; diff --git a/src/w32.c b/src/w32.c index 999541b25ad..cd95f60e84f 100644 --- a/src/w32.c +++ b/src/w32.c @@ -840,13 +840,13 @@ static struct group dflt_group = 0, }; -int +unsigned getuid () { return dflt_passwd.pw_uid; } -int +unsigned geteuid () { /* I could imagine arguing for checking to see whether the user is @@ -855,20 +855,20 @@ geteuid () return getuid (); } -int +unsigned getgid () { return dflt_passwd.pw_gid; } -int +unsigned getegid () { return getgid (); } struct passwd * -getpwuid (int uid) +getpwuid (unsigned uid) { if (uid == dflt_passwd.pw_uid) return &dflt_passwd; @@ -2894,7 +2894,7 @@ get_rid (PSID sid) #endif struct w32_id { - int rid; + unsigned rid; struct w32_id *next; char name[GNLEN+1]; unsigned char sid[FLEXIBLE_ARRAY_MEMBER]; @@ -2903,7 +2903,7 @@ struct w32_id { static struct w32_id *w32_idlist; static int -w32_cached_id (PSID sid, int *id, char *name) +w32_cached_id (PSID sid, unsigned *id, char *name) { struct w32_id *tail, *found; @@ -2926,7 +2926,7 @@ w32_cached_id (PSID sid, int *id, char *name) } static void -w32_add_to_cache (PSID sid, int id, char *name) +w32_add_to_cache (PSID sid, unsigned id, char *name) { DWORD sid_len; struct w32_id *new_entry; @@ -2953,7 +2953,7 @@ w32_add_to_cache (PSID sid, int id, char *name) static int get_name_and_id (PSECURITY_DESCRIPTOR psd, const char *fname, - int *id, char *nm, int what) + unsigned *id, char *nm, int what) { PSID sid = NULL; char machine[MAX_COMPUTERNAME_LENGTH+1]; @@ -3837,8 +3837,8 @@ system_process_attributes (pid) DWORD blen = 0; TOKEN_USER user_token; TOKEN_PRIMARY_GROUP group_token; - int euid; - int egid; + unsigned euid; + unsigned egid; DWORD sess; PROCESS_MEMORY_COUNTERS mem; PROCESS_MEMORY_COUNTERS_EX mem_ex; -- 2.30.2